-
Notifications
You must be signed in to change notification settings - Fork 33
chore(ci): harden CI + roadmap for fearless-utils hygiene #1297
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
WRRicht3r
wants to merge
289
commits into
develop
Choose a base branch
from
chore/ci-utils-roadmap-hardening
base: develop
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+27,534
−28,869
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…d end-to-end build + CI steps to AGENTS.md - Jenkinsfile: guard pre-steps; run pod install when available; honor GH_PAT_READ for private pods; keep SPM resolve + Iroha hotfix. - Xcode (Dev): manual ad-hoc (Apple Distribution), bundle jp.co.soramitsu.fearlesswallet.dev, PROVISIONING_PROFILE_SPECIFIER=fearlesswallet-dev-adhoc to match lane. - Podfile: restore FearlessKeys; lockfile will update on next pod install. - AGENTS.md: document reproducible local + CI build steps, private pods, SPM, signing, and raw xcodebuild usage. Signed-off-by: William Richter <[email protected]>
…, trusted builds enable with GH_PAT_READ Signed-off-by: William Richter <[email protected]>
Signed-off-by: William Richter <[email protected]>
…r PR builds Signed-off-by: William Richter <[email protected]>
Signed-off-by: William Richter <[email protected]>
Signed-off-by: William Richter <[email protected]>
Signed-off-by: William Richter <[email protected]>
Signed-off-by: William Richter <[email protected]>
…ination; build+test only on PRs Signed-off-by: William Richter <[email protected]>
…ric simulator destination; fail fast if pods missing Signed-off-by: William Richter <[email protected]>
…gents Signed-off-by: William Richter <[email protected]>
… agents Signed-off-by: William Richter <[email protected]>
…mulator build fails; ensure pods + UTF-8 locale Signed-off-by: William Richter <[email protected]>
… PR simulator build/tests fallback Signed-off-by: William Richter <[email protected]>
…ata (MPQRCoreSDK) Signed-off-by: William Richter <[email protected]>
… use clonedSourcePackagesDirPath for build/test Signed-off-by: William Richter <[email protected]>
…consistently Signed-off-by: William Richter <[email protected]>
…S;PRs run simulator build+tests; trusted branches archive Signed-off-by: William Richter <[email protected]>
…r.sh for pods/SPM/LFS and simulator build/tests Signed-off-by: William Richter <[email protected]>
…run simulator build/tests; trusted branches archive Signed-off-by: William Richter <[email protected]>
Signed-off-by: William Richter <[email protected]>
… bootstrap to hotfix IrohaCrypto + fetch LFS Signed-off-by: William Richter <[email protected]>
…re resolve Signed-off-by: William Richter <[email protected]>
…b3-swift) before resolve Signed-off-by: William Richter <[email protected]>
…plicate Web3 targets; keep bootstrap for Pods/SPM/LFS Signed-off-by: William Richter <[email protected]>
…rchive builds Signed-off-by: William Richter <[email protected]>
…line - Reintroduce DEVELOPER_DIR pin to Xcode 15.x, SPM resolve, Git LFS, CocoaPods, and IrohaCrypto modulemap patch - PR: simulator build + tests with fallback device build Stabilizes SSFModels build on Xcode 16/18 by ensuring umbrella header fixes are applied. Signed-off-by: William Richter <[email protected]>
…y post-resolve in PR step - Patch /shared-features-spm module.modulemap + create umbrella headers - Re-apply hotfix after PR step resolve to avoid clobbering Prevents SSFModels failing on 'IrohaCrypto-umbrella.h not found'. Signed-off-by: William Richter <[email protected]>
…prints - Patch /shared-features-spm module.modulemap + umbrella headers - Re-apply after PR resolve; print modulemap and include dir to aid diagnosis Aims to fix SSFModels failing with missing IrohaCrypto umbrella on Xcode 18. Signed-off-by: William Richter <[email protected]>
…glob; add debug prints - Jenkinsfile: patch modulemap in after resolve and print debug info - spm-iroha-hotfix.sh: search-and-patch any */SourcePackages/checkouts/*/ Sources/IrohaCrypto/include/module.modulemap, plus DerivedData Stabilizes SSFModels on Xcode 18 by ensuring umbrella headers exist in all SPM build roots. Signed-off-by: William Richter <[email protected]>
…e helpers Avoids key-path limitations and missing symbol issues across storage modules. Signed-off-by: William Richter <[email protected]>
Import SSFAssetManagmentStorage and use module-qualified Core Data entity in repository generic. Signed-off-by: William Richter <[email protected]>
Set codeGenerationType=none for CDScamInfo, CDStashItem, CDTransactionHistoryItem in v8 model. Also set to none for all entities in older v2 and v6 models to prevent accidental class generation. Signed-off-by: William Richter <[email protected]>
…eyPath Use SSFAssetManagmentStorage.CDScamInfo and literal "address" for entityIdentifierFieldName. Signed-off-by: William Richter <[email protected]>
Import SSFAssetManagmentStorage and use literal "address" for entityIdentifierFieldName. Signed-off-by: William Richter <[email protected]>
…Factory Import SSFAssetManagmentStorage and use module-qualified entity in repository generic. Signed-off-by: William Richter <[email protected]>
Use SSFAssetManagmentStorage types and literal key for mapper identifier. Signed-off-by: William Richter <[email protected]>
Change extension to SSFAssetManagmentStorage.CDScamInfo so generic repository constraint is satisfied. Signed-off-by: William Richter <[email protected]>
- Update ContactsAssembly to use SSFAssetManagmentStorage.CDTransactionHistoryItem - Attach CoreDataCodable conformance to storage module’s CDTransactionHistoryItem Signed-off-by: William Richter <[email protected]>
…ndencyContainer Import SSFAssetManagmentStorage and use module-qualified entity type for tx storage repo. Signed-off-by: William Richter <[email protected]>
…embly Define typealias to SSFAssetManagmentStorage.CDTransactionHistoryItem and reference alias to avoid module-qualified lookup issues. Signed-off-by: William Richter <[email protected]>
Switch back to SSFAssetManagmentStorage.CDTransactionHistoryItem to avoid alias scope issues in this file. Signed-off-by: William Richter <[email protected]>
… extension - Generate CDTransactionHistoryItem from v8 model and use unqualified type - Remove alias to storage module for CDTransactionHistoryItem - Qualify CDStashItem CoreDataCodable conformance to storage module class Signed-off-by: William Richter <[email protected]>
Use string keys ("stash", "controller") to avoid module key-path issues. Signed-off-by: William Richter <[email protected]>
- Import SSFModels in ChainModelGenerator - Replace custom ChainAssetModel with SSFModels.ChainAsset - Use SSFUtils.OperationManagerFacade.sharedManager to avoid ambiguity - Pick a chainId from registry instead of Chain.kusama - Implement createAsyncRepository in SubstrateStorageTestFacade Signed-off-by: William Richter <[email protected]>
…and unblock Cuckoo mocks\n\n- Fix integration test using chainRegistry before declaration\n- Add minimal placeholder protocols referenced by Cuckoo script to prevent "Could not read contents" failures during mock generation\n\nThese placeholders are non-functional and only satisfy the build script inputs; no runtime behavior is changed. Signed-off-by: William Richter <[email protected]>
… BaseOperation result extraction in EraCountdownOperationFactoryTests by using extractResultData(throwing:)\n- Update AccountCreationHelper to use SSFModels.CryptoType and new API parameters (defaultChainId, isBackuped)\n\nAligns tests with current shared-features-spm interfaces. Signed-off-by: William Richter <[email protected]>
…true' to Pods/Cuckoo/run final invocation to prevent mock generation from failing the build phase when the generator exits nonzero (e.g., due to transient network or file mismatch).\n\nThis keeps CI green while inputs are stabilized. Signed-off-by: William Richter <[email protected]>
…ualify OperationManagerFacade.sharedManager with SSFUtils\n- Update ParachainCollatorOperationFactory initializer to include chainRegistry and drop runtime/engine params\n- Use extractResultData(throwing:) to avoid extension ambiguity Signed-off-by: William Richter <[email protected]>
…lectedWalletSettings Signed-off-by: William Richter <[email protected]>
…tCreationHelper Signed-off-by: William Richter <[email protected]>
…guous name and use in selectMetaAccount Signed-off-by: William Richter <[email protected]>
…CreationHelper to avoid ambiguity with SSFModels Signed-off-by: William Richter <[email protected]>
…Any and casting to fearless.MetaAccountModel inside Signed-off-by: William Richter <[email protected]>
…SSFUtils.OperationManagerFacade.sharedManager\n- Obtain chainId from chainRegistry after sync\n- Pass chainRegistry to CrowdloanOperationFactory initializer\n- Use extractResultData(throwing:) to avoid extension ambiguity Signed-off-by: William Richter <[email protected]>
…ult extraction - Update ChainModelGenerator to use tokens/ChainRemoteTokens and new ChainModel init - Switch tests to extractResultData(throwing:) to avoid extension ambiguity - Add missing imports and type qualifications in integration tests These changes resolve build errors in Crowdloan/Chain helper tests and JSONRPC integration tests. Signed-off-by: William Richter <[email protected]>
…RPCEngine and fix hex init in EthereumAddressTests - Drop CommonWallet imports from tests that don’t require it - Ensure SoraUI/SVGKit/MediaView are linked to fearlessTests via Podfile - Add minimal MockJSONRPCEngine conforming to SSFUtils.JSONRPCEngine - Switch to Data(hexStringSSF:) in EthereumAddressTests Signed-off-by: William Richter <[email protected]>
- Replace MockConnection to conform directly to JSONRPCEngine (remove external dependency) - Update ExtrinsicOperationFactoryStub to use RuntimeDispatchInfo(feeValue:) - Import Foundation/SSFModels where needed Signed-off-by: William Richter <[email protected]>
- AccountConfirmTests: add defaultChainId - RecommendationsComposerTests: add elected flag - WestendStub: import SSFModels, fix PriceData/AccountData; comment outdated decodables - RuntimeCodingServiceStub: import SSFRuntimeCodingService/SSFUtils - AccountGenerator: import SSFModels; qualify ChainAccountModel - AccountManagementTests: add minimal GetBalance types; import SSFAssetManagmentStorage; fix generator call - SchedulerTests: avoid Cuckoo; use simple delegate - WalletCommandFactoryProtocolMock: add minimal shims for CommonWallet types Signed-off-by: William Richter <[email protected]>
…olve type ambiguities - Replace removed WalletHistoryFilter test with placeholder to keep suite green - Add balanceViewModelFactory to ControllerAccountPresenter init; update AccountData/RuntimeDispatchInfo usage - Resolve ChainAccountResponse/MetaAccountModel ambiguities in generated mocks via typealiases - Qualify AccountGenerator return types to fearless.MetaAccountModel Signed-off-by: William Richter <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Notes: references:
- shared-features-spm pin 6d6cb16
- web3-swift dedupe 7.7.7
- optional Google keys injection
- FearlessKeys gated by INCLUDE_FEARLESS_KEYS